Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor evaluation of inbound criteria of join task in orquesta workflow #4898

Merged
merged 2 commits into from
Apr 1, 2020

Conversation

m4dcoder
Copy link
Contributor

@m4dcoder m4dcoder commented Apr 1, 2020

Update orquesta dependency to version that refactored evaluation of inbound criteria for join task on inbound task completion and not on inbound task transition. This allows for different use cases such as multiple task transition from the same inbound task, inbound task transition on completion or inbound task transition on failure. Include additional check in the state machine to detect case where there is an unreachable join task where inbound criteria is partially satisfied but all inbound tasks have already completed and it will never be satisfied.

…flow

Update orquesta dependency to version that refactored evaluation of inbound criteria for join task on inbound task completion and not on inbound task transition. This allows for different use cases such as multiple task transition from the same inbound task, inbound task transition on completion or inbound task transition on failure. Include additional check in the state machine to detect case where there is an unreachable join task where inbound criteria is partially satisfied but all inbound tasks have already completed and it will never be satisfied.
@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines. Very easy to review. label Apr 1, 2020
@m4dcoder m4dcoder requested a review from blag April 1, 2020 00:34
@m4dcoder m4dcoder added this to the 3.2.0 milestone Apr 1, 2020
@blag
Copy link
Contributor

blag commented Apr 1, 2020

Waiting on #4895 to fix the build.

@blag blag merged commit ab6dd77 into master Apr 1, 2020
@blag blag deleted the fix-orquesta-join branch April 1, 2020 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S PR that changes 10-29 lines. Very easy to review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants